Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[styled-engine] Add support for goober #27776

Closed

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Aug 15, 2021

I have pushed the exploration further than I could in the past.

The incentive is https://bundlephobia.com/package/goober, likely 3 kB gzipped end-to-end, instead of +10 kB with emotion. You can see it used on https://react-hot-toast.com/.

@mui-pr-bot
Copy link

mui-pr-bot commented Aug 15, 2021

Bundle size will be reported once Azure build #31807 finishes.

Generated by 🚫 dangerJS against be825e1

@oliviertassinari oliviertassinari force-pushed the styled-engine-goober branch 3 times, most recently from 64893f6 to c6f812f Compare August 15, 2021 19:21
@oliviertassinari
Copy link
Member Author

oliviertassinari commented Aug 15, 2021

It seems that making it work would require a non-negligible amount of effort, there are a few bugs that are time-consuming to get to the bottom of. I'm not too far but I'm closing, for now, we could revisit in the future. It seems to have potential :D

Capture d’écran 2021-08-15 à 22 23 43

The value could be for third-party libraries that want to minimize the style overhead, e.g. cristianbote/goober#180 (comment) by @iamhosseindhv. We might also want to see if developers do the effort to switch from emotion to styled-components. Two graphs to keep an eye on:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: styled-engine Specific to @mui/styled-engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants